Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhancement: xmllint xsd support added #3665

Merged
merged 7 commits into from
Jun 22, 2024

Conversation

TommyE123
Copy link
Contributor

xmllint add xsd support

Fixes #3363

Proposed Changes

  1. Added xsd to the xmllint descriptor to pickup the xsd files when running a scan.
  2. Added some good, bad and fix test files to check on builds.
  3. Updated the url to the xmllint configuration.

Testing

image

@echoix
Copy link
Collaborator

echoix commented Jun 16, 2024

As always, well explained and seems perfect like it. An unrelated failure was there from sfdx, so I'm rerunning it

@echoix echoix enabled auto-merge (squash) June 16, 2024 21:01
@bdovaz bdovaz disabled auto-merge June 17, 2024 05:13
@echoix
Copy link
Collaborator

echoix commented Jun 20, 2024

I'm still trying to work around the CVE reported by trivy. It seems the 1.21.11 go stdlib isn't published on alpine packages, so I'm looking for upgrading the base image to alpine3.20

@echoix echoix merged commit 4f86158 into oxsecurity:main Jun 22, 2024
6 checks passed
@TommyE123 TommyE123 deleted the xmllint_XSD_added branch June 23, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Xmllint: also lint xsd files
2 participants